Skip to content

Conversation

@clarmso
Copy link
Collaborator

@clarmso clarmso commented Nov 3, 2025

📜 Tickets

Jira ticket
Github issue

💡 Description

Let me use this PR to see which unit tests and smoke tests that need updates.

🎥 Demos

Before After
Demo

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If adding or modifying strings, I read the guidelines and will request a string review from l10n
  • If needed, I updated documentation and added comments to complex code

@clarmso clarmso added the Do Not Merge ⛔️ This issue is a work in progress and is not ready to land label Nov 3, 2025
@mobiletest-ci-bot
Copy link

Messages
📖 Project coverage: 37.17%

💪 Quality guardian

1 tests files modified. You're a champion of test coverage! 🚀

🧹 Tidy commit

Just 2 file(s) touched. Thanks for keeping it clean and review-friendly!

✅ Per-file coverage

All changed files meet the threshold of 35.0%.

Client.app: Coverage: 36.96

File Coverage
SearchEngineFlagManager.swift 100.0%

Generated by 🚫 Danger Swift against e8b1796

@clarmso clarmso force-pushed the cs/MTE-4982-fix-ui-tests branch from e8b1796 to fab15c8 Compare November 5, 2025 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Merge ⛔️ This issue is a work in progress and is not ready to land

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants